Repository navigation
Added AzLinux-dnf support for RedisPackageInstallation - #417
Rakesh (RakeshwarK) wants to merge 22 commits into
Conversation
Signed-off-by: Rakesh <153008248+RakeshwarK@users.noreply.github.com>
| } | ||
| }, | ||
| { | ||
| "Type": "WgetPackageInstallation", |
There was a problem hiding this comment.
Have you tested the changes in ubuntu?
There was a problem hiding this comment.
There seems to be some issue with Redis, entering protected mode in ubuntu even without these changes, but the changes were running fine on AZ Linux.
|
|
||
| } | ||
|
|
||
| private async Task InstallOnAzLinuxAsync(EventContext telemetryContext, CancellationToken cancellationToken) |
There was a problem hiding this comment.
have you tried tdnf?
There was a problem hiding this comment.
tdnf also works, is it preferred!?
There was a problem hiding this comment.
on AzLinux yes, on RHEL still use dnf. You don;t have to change it right now, you could add a comment that tdnf also works. I heard AzLinux eventually wants to move to tdnf entirely
|
|
||
| } | ||
|
|
||
| private async Task InstallOnAzLinuxAsync(EventContext telemetryContext, CancellationToken cancellationToken) |
There was a problem hiding this comment.
I believe this would also work on RHEL/Fedora distros?
There was a problem hiding this comment.
Yes, but I haven't tested these cases. Should I also include RHEL/Fedora distros?
There was a problem hiding this comment.
I think so, Redis is important enough that we need more than ubuntu and AzLinux
2 participants
No description provided.